Skip to content

Add docs for Azure Federated Identity via Dapr Sentry OIDC #4650

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 18 commits into from
Sep 5, 2025
Merged

Add docs for Azure Federated Identity via Dapr Sentry OIDC #4650

merged 18 commits into from
Sep 5, 2025

Conversation

jjcollinge
Copy link
Contributor

@jjcollinge jjcollinge commented May 11, 2025 •
edited
Loading

Add docs for using the work implemented in the Dapr Sentry OIDC support PR to access resources in Azure via a federated identity credential.

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

Issue reference

@jjcollinge jjcollinge changed the base branch from v1.15 to v1.16 May 11, 2025 07:43
@jjcollinge jjcollinge mentioned this pull request May 11, 2025
Merged
7 tasks
@jjcollinge jjcollinge marked this pull request as ready for review May 12, 2025 11:36
@jjcollinge jjcollinge requested review from a team as code owners May 12, 2025 11:36
@msfussell msfussell added the waiting-on-code-pr The code PR needs to be merged before the docs are updated label Jun 3, 2025
@jjcollinge
Update helm values
7c0fb2c 
Signed-off-by: Jonathan Collinge <[email protected]>
@jjcollinge
Remove jwt audiences from configuration
d2ca86a 
Signed-off-by: Jonathan Collinge <[email protected]>
@jjcollinge
Add warning
3dc1950 
Signed-off-by: Jonathan Collinge <[email protected]>
@jjcollinge
Merge branch 'v1.16' into jjcollinge/azure-oidc
cf03e22
@jjcollinge
Update args and annotation
4bee790 
Signed-off-by: Jonathan Collinge <[email protected]>
@jjcollinge
Fix localized URLs
dad1a5e 
Signed-off-by: Jonathan Collinge <[email protected]>
@nelson-parente nelson-parente mentioned this pull request Jul 25, 2025
Merged
3 tasks
marcduiker
marcduiker approved these changes Jul 25, 2025
View reviewed changes
Copy link
Contributor

@marcduiker marcduiker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msfussell msfussell added this to the 1.16 milestone Jul 30, 2025
@jjcollinge
Add comment to increase init timeout
79a311a 
Signed-off-by: Jonathan Collinge <[email protected]>
@msfussell msfussell removed the waiting-on-code-pr The code PR needs to be merged before the docs are updated label Aug 29, 2025
msfussell
msfussell reviewed Aug 29, 2025
View reviewed changes
@msfussell
Update daprdocs/content/en/developing-applications/integrations/Azure…
6df8c3c 
…/azure-authentication/authenticating-azure.md

Signed-off-by: Mark Fussell <[email protected]>
@msfussell
Update daprdocs/content/en/developing-applications/integrations/Azure…
756a3bb 
…/azure-authentication/authenticating-azure.md

Signed-off-by: Mark Fussell <[email protected]>
@msfussell
Update daprdocs/content/en/developing-applications/integrations/Azure…
740aba1 
…/azure-authentication/authenticating-azure.md

Signed-off-by: Mark Fussell <[email protected]>
@msfussell
Update daprdocs/content/en/developing-applications/integrations/Azure…
8271611 
…/azure-authentication/authenticating-azure.md

Signed-off-by: Mark Fussell <[email protected]>
@msfussell
Update daprdocs/content/en/developing-applications/integrations/Azure…
5a926bb 
…/azure-authentication/authenticating-azure.md

Signed-off-by: Mark Fussell <[email protected]>
@msfussell
Copy link
Member

@alicejgibbons - PTAL and review

MyMirelHub
MyMirelHub approved these changes Sep 4, 2025
View reviewed changes
Copy link
Contributor

@MyMirelHub MyMirelHub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

atrauzzi
atrauzzi approved these changes Sep 5, 2025
View reviewed changes
Copy link
Contributor

@atrauzzi atrauzzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Good to call out that it has to be public, and to make sure the aud is set up right 👍

@msfussell msfussell merged commit a9d5e8b into dapr:v1.16 Sep 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msfussell msfussell msfussell left review comments

@marcduiker marcduiker marcduiker approved these changes

@nelson-parente nelson-parente nelson-parente approved these changes

@alicejgibbons alicejgibbons Awaiting requested review from alicejgibbons

+2 more reviewers

@atrauzzi atrauzzi atrauzzi approved these changes

@MyMirelHub MyMirelHub MyMirelHub approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@jjcollinge jjcollinge

Labels
None yet
Projects
None yet
Milestone
1.16
Development

Successfully merging this pull request may close these issues.
6 participants
@jjcollinge @msfussell @atrauzzi @marcduiker @MyMirelHub @nelson-parente